Skip to content

Add type annotations and mypy CI to flatdata-py and flatdata-generator#262

Merged
VeaaC merged 23 commits intoheremaps:masterfrom
VeaaC:type_annotations
Apr 27, 2026
Merged

Add type annotations and mypy CI to flatdata-py and flatdata-generator#262
VeaaC merged 23 commits intoheremaps:masterfrom
VeaaC:type_annotations

Conversation

@VeaaC
Copy link
Copy Markdown
Collaborator

@VeaaC VeaaC commented Apr 24, 2026

Adds type annotations to all source functions/methods in both Python projects and
enforces them via mypy in CI.

What changed

Type annotations added.

mypy configuration (both pyproject.toml): 9 strictness flags enabled —
disallow_untyped_defs, check_untyped_defs, disallow_any_generics,
warn_return_any, warn_unused_configs, warn_redundant_casts,
warn_unused_ignores, no_implicit_optional, strict_equality. Namespace
package settings for the shared flatdata/ namespace.

CI: mypy steps added to generator.yml and py.yml, running after tests.

PEP 561: py.typed markers in flatdata/lib/ and flatdata/generator/.

@VeaaC VeaaC force-pushed the type_annotations branch 3 times, most recently from 76e1c61 to 4268fd0 Compare April 24, 2026 10:00
VeaaC and others added 23 commits April 27, 2026 15:19
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
…ping

When archive_name is a str, the method returns tuple[ModuleType, Any].
When archive_name is None (default), it returns just ModuleType.
Without overloads, mypy sees the union return type and rejects tuple
unpacking at call sites (attr-defined: Module has no __iter__).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
Signed-off-by: Christian Vetter <christian.vetter@here.com>
@VeaaC VeaaC force-pushed the type_annotations branch from 57066a0 to b3e3414 Compare April 27, 2026 13:19
@VeaaC VeaaC merged commit 9caaf9d into heremaps:master Apr 27, 2026
9 checks passed
@VeaaC VeaaC deleted the type_annotations branch April 27, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant